/approve thanks !

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: arkodg, zirain

The full list of commands accepted by this bot can be found here.

The pull request process is described here

I think this is okay, as it only runs after the test has been completed. It's super important that we don't allow hooks before, as we don't want the behavior of the conformance tests to be able to be modified (or else we can't be sure that the same behavior is executed for every implementation, defeating the point of conformance).

I won't LGTM though, as I'd like @mlavacca to confirm he's okay with this. Holding for his review.

/hold

I want to clearify that, we want this to run dump function for debugging a flaky test.
That's why this was only added to post-test.

Thanks @zirain!

/lgtm

FYI @spencerhance @SinaChavoshi in case this is helpful on inference extension tests

I think this is okay, as it only runs after the test has been completed. It's super important that we don't allow hooks before, as we don't want the behavior of the conformance tests to be able to be modified (or else we can't be sure that the same behavior is executed for every implementation, defeating the point of conformance).

I won't LGTM though, as I'd like @mlavacca to confirm he's okay with this. Holding for his review.

/hold

Missed this, will leave the hold in place for @mlavacca.